Can you update the JSDoc types in DataTable.svelte (Lines 5, 6, 10) and run yarn build:docs?

According to the docs, only line 10 needs update: the "display" method receives the row object only for cells, not headers, am I right?

Just rebased & pushed an update

I believe that the display is reflected in the DataTableNonEmptyHeader and DataTableEmptyHeader interfaces.

When testing the types, I see this:

Repro: try using row in the type test for DataTable:

I don't see a reason to make this change on headers: row object should not be accessible when displaying headers 🤔

This means two things:

• The row object should be nullable for headers, and non-null for rows, I don't know how to do that
• The display function is overriden from the headers object, that's probably the cause of these typing issues:
  

  carbon-components-svelte/src/DataTable/DataTable.svelte

  Lines 200 to 204 in aae2dd4

  	rows[row.id] = headerKeys.map((key, index) => ({
  	key,
  	value: resolvePath(row, key),
  	display: headers[index].display,
  	}));

I think we should change the behavior for that.

I just pushed a fix to add row?: ... to header-related display functions, and kept row: ... for cell-related.

I'm not sure the rest of the PR is okay, feel free to make more comments 👌

Thank you @metonym !

Released in v0.81.0